Skip to content

Sync eng/common directory with azure-sdk-tools for PR 4311#1202

Merged
azure-sdk merged 1 commit intomainfrom
sync-eng/common-FixVerifyLinks500Error-4311
Oct 6, 2022
Merged

Sync eng/common directory with azure-sdk-tools for PR 4311#1202
azure-sdk merged 1 commit intomainfrom
sync-eng/common-FixVerifyLinks500Error-4311

Conversation

@azure-sdk
Copy link
Collaborator

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#4311 See eng/common workflow

We have hit a few 50x errors while pulling a page to get
links and that has caused the rest of the link checking to
terminate early. To fix that we switching to LogError which will
output an error in devops instead of Write-Error which terminates
immediately. We also add some retry count to the page retrieval
and cache file retrieval to help with these transitent 50x issues.
@azure-sdk azure-sdk added Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system. labels Oct 6, 2022
@azure-sdk azure-sdk merged commit 0c10cbe into main Oct 6, 2022
@azure-sdk azure-sdk deleted the sync-eng/common-FixVerifyLinks500Error-4311 branch October 6, 2022 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants